Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove surelog libs out of own folder, use soversion #3828

Merged
merged 2 commits into from
Sep 10, 2023
Merged

Remove surelog libs out of own folder, use soversion #3828

merged 2 commits into from
Sep 10, 2023

Conversation

timkpaine
Copy link
Collaborator

Parallel of chipsalliance/UHDM#1002

  • install into lib instead of lib/surelog, python files moved to lib/surelog-python (open to other options, should probably move to a real python package)
  • use SOVERSION so that lib/libsurelog.so is now a symlink, and we install a versioned lib/libsurelog.so..

CMakeLists.txt Outdated Show resolved Hide resolved
@alaindargelas
Copy link
Collaborator

yosys-systenverilog-plugin test is faiing here, can't merge.

@timkpaine
Copy link
Collaborator Author

timkpaine commented Sep 3, 2023

yosys-systenverilog-plugin test is faiing here, can't merge.

I can take a look, though it seems unrelated to this PR. As part of my Mac PR on the yosys plugins, I've removed all the custom conda stuff so maybe this will be more reliable/easily maintainable in the nearish future.

@alaindargelas
Copy link
Collaborator

alaindargelas commented Sep 3, 2023

@timkpaine Looks like this is the root cause of the failure of the plugin:
https://github.com/chipsalliance/Surelog/actions/runs/6059758147
The run immediately before that on main was still working

@timkpaine
Copy link
Collaborator Author

@alaindargelas pretty sure its this: chipsalliance/synlig@ec729d5

@kbieganski

@alaindargelas
Copy link
Collaborator

@timkpaine correct. So you can make this PR official.

@timkpaine timkpaine marked this pull request as ready for review September 9, 2023 22:52
@alaindargelas alaindargelas merged commit d708b80 into chipsalliance:master Sep 10, 2023
61 checks passed
@timkpaine timkpaine deleted the tkp/install branch September 10, 2023 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants